feat/32 - mapa #49
feat/32 - mapa #49
Conversation
AitorRD
added
code
Signed-off-by: GitHub <[email protected]>
Deploying with
|
| Latest commit: |
527fb3e
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://2ffbca44.ocial-app.pages.dev |
| Branch Preview URL: | https://feat-32-mapa.ocial-app.pages.dev |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
Se podría cambiar el Status del Hotspot a Safe, ya que el navegador te pregunta el permiso de toma de ubicación sin el permiso no puede hacerlo. Por lo que cambiaría el estado a Safe para que no de problemas en el rebase. @ispp-2324-ocial/qa @ispp-2324-ocial/frontend
|
src/components/LMap.vue
Outdated
|
|
||
| const userLocationMarker = marker([latitude, longitude], { icon: userIcon }).addTo(mapInstance); | ||
|
|
||
| userLocationMarker.bindPopup('Tu ubicación'); |
There was a problem hiding this comment.
Mira el vídeo que he hecho sobre como traducir el texto y que he mandado por Discord (ahora añadiré todos los vídeos al README para que estén más a mano).
There was a problem hiding this comment.
He resuelto este cambio en el commit.
There was a problem hiding this comment.
El problema sigue siendo el error en el async
AitorRD
force-pushed
the
feat/32-mapa
branch
2 times, most recently
from
ca3035b to
3bfed08
Compare
src/components/LMap.vue
Outdated
| @@ -95,6 +95,7 @@ function createMapLayer(): void { | |||
| } | |||
| } | |||
| }, | |||
| // eslint-disable-next-line promise/prefer-await-to-callbacks | |||
There was a problem hiding this comment.
Esta se ha añadido para corregir el error de lint que daba en esta función, puesto que no era un error per se, simplemente te recomendaba hacer uso de promises y async lo cual acarreaba más warnings de los que se podía solucionar.
There was a problem hiding this comment.
@AitorRD El problema era efectivamente del linter, sin embargo, mágicamente le ha gustado al añadir la opción de HighAccuracy (la vi el otro día en la documentación de MDN y se me olvidó decírtelo, pero para nuestro caso es estrictamente necesario) 😁.
There was a problem hiding this comment.
Mira si todo funciona bien con mis últimos cambios (se te olvidó sacar el instanciamiento del mapa fuera del callback, y he eliminado algunos console.log, es lo único extra además de añadir el highAccuracy que he cambiado).
Y por último (y ya con esto prometo que va todo para dentro 🙏), en vez de utilizar console.error para los errores, utiliza el notificador que ha hecho @paupenfer1, así si el usuario no ha dado permisos de acceso a la ubicación o ha habido cualquier tipo de error se lo mostramos, y no se lo ocultamos de forma opaca.
Co-authored-by: Fernando Fernández <[email protected]>
Co-authored-by: Fernando Fernández <[email protected]> Signed-off-by: GitHub <[email protected]>
src/components/LMap.vue
Outdated
| if (!mapViewSet) { | ||
| mapInstance!.setView([latitude, longitude], 15); | ||
| mapViewSet = true; | ||
| } |
There was a problem hiding this comment.
Esta lógica está mal, en sucesivas iteraciones siempre evaluará a false y no actualizará la posición en el mapa.
Lo arreglo en mi push para terminar ya este PR, lo he testeado a mano, si hay cualquier otro problema manda un PR con un hotfix
ferferga
force-pushed
the
feat/32-mapa
branch
2 times, most recently
from
3c74602 to
7a5a0a1
Compare
Co-authored-by: Fernando Fernández <[email protected]> Signed-off-by: GitHub <[email protected]>
Signed-off-by: GitHub <[email protected]>
|
Resuelve la isse #32